Skip to content

Conversation

gareth-nx
Copy link
Contributor

These edits to the README attempt to offer simple guidance on compiling stdlib with optimization flags.

In this discussion here it became clear that not everyone understands how to apply compiler optimizations to stdlib, although the discussion shows that is quite important for performance.

I think some simple pointers would help users who do not have much experience with build tools (like me!).

@awvwgk awvwgk added build: cmake Issue with stdlib's CMake build files build: make Issue with stdlib's manual makefile build documentation Improvements or additions to documentation reviewers needed This patch requires extra eyes labels Sep 19, 2021
Copy link
Member

@awvwgk awvwgk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for sharing, looks good to me.

Copy link
Member

@jvdp1 jvdp1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for these additions. I approve pending minor comments.

gareth-nx and others added 2 commits September 21, 2021 08:53
Copy link
Member

@milancurcic milancurcic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @gareth-nx!

@milancurcic milancurcic merged commit 5614737 into fortran-lang:master Sep 21, 2021
@awvwgk awvwgk removed the reviewers needed This patch requires extra eyes label Sep 25, 2021
@gareth-nx gareth-nx deleted the pr/readme_updates branch October 7, 2021 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build: cmake Issue with stdlib's CMake build files build: make Issue with stdlib's manual makefile build documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants